-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add device integration page #1248
Conversation
Images automagically compressed by Calibre's image-actions ✨ Compression reduced images by 14.2%, saving 4.60 KB.
619 images did not require optimisation. |
@iskerrett @MarianRaphael should I add a link to this page under the product menu? or does it belong somewhere else? |
I would like to it from the Hardware Partner section on the https://flowfuse.com/partners/ page. Maybe add a button called 'Device Integration Details' |
@iskerrett I just added the button, I used "CERTIFY YOUR HARDWARE" to make it similar to the title they will find in the |
@iskerrett @MarianRaphael I just added the FlowFuse certified device logo. Once we've replaced the placeholder text in the "certification" section, it will be fully ready for review. These are the pages that have been updated: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks excellent, I suggested a change for the placeholder text.
<ul class="flex flex-row justify-center w-full items-center"> | ||
<li class="pb-6 hover:transition-opacity hover:opacity-75 transition ease-in duration-300"> | ||
<a href="/book-demo/" class="inline-block ff-btn ff-btn--primary uppercase w-[218px] m-auto text-center"> | ||
<span>BOOK A MEETING</span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Book a Meeting button goes to Zack calendar which I don't think is correct. @MarianRaphael will you be doing these meetings?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just replaced the link, in the future I might change it to be a pop-up
Description
Created the
/certify-hardware/
pageRelated Issue(s)
closes #1118
Checklist